feat: support transpiled transformers #11193
Conversation
|
Deploy preview for jestjs ready! Built without sensitive environment variables with commit 2411699 |
|
Deploy preview for jestjs ready! Built without sensitive environment variables with commit 76b2b75 |
Codecov Report
@@ Coverage Diff @@
## master #11193 +/- ##
=======================================
Coverage 64.28% 64.28%
=======================================
Files 307 307
Lines 13458 13458
Branches 3285 3285
=======================================
Hits 8652 8652
Misses 4100 4100
Partials 706 706
Continue to review full report at Codecov.
|
| @@ -255,7 +255,7 @@ class ScriptTransformer { | |||
| let transformer: Transformer; | |||
|
|
|||
| try { | |||
| transformer = require(transformPath); | |||
| transformer = interopRequireDefault(require(transformPath)).default; | |||
There was a problem hiding this comment.
@SimenB This incorrectly assumes that a Jest transformer does not use export const createTransformer instead of export default { createTransformer }.
This issue was caused by jestjs/jest#11193
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
Vaguely related to #8810. We don't transpile at runtime, but pre-transpiled modules should load fine
Test plan
Green CI